Skip to content

Conversation

@jakirkham
Copy link
Member

hcho3 and others added 30 commits June 28, 2024 17:40
automerged PR by conda-forge/automerge-action
This directory doesn't exist in XGBoost 2.1.0.
Co-authored-by: jakirkham <[email protected]>
These precede the `libmamba` solver becoming the default in Conda. Very
likely they are unneeded now (and may be causing issues). So drop them
to see if it helps. Can always readd one-by-one if needed.
automerged PR by conda-forge/automerge-action
In some cases `pin_subpackage` was only added to `requirements/run`. In
others it may be in `requirements/host` too. This consistently adds it
to both. May help build tools recognize the dependency order when
starting builds (and thus more consistently create working test
environments).
jakirkham and others added 26 commits July 23, 2024 23:31
Co-authored-by: jakirkham <[email protected]>
The linter does not like having `noarch: python` at the top-level with
selectors. Rewriting the recipe without selectors is a substantial lift
and not worth it given the move towards rattler-build.

However if we don't have `noarch` or a `skip` at the top-level, a full
matrix of Python versions is built out for `xgboost` (despite these
packages being redundant).

To clean this up, we skip any Python version except the minimum one that
XGBoost supports. This ensure we build for exactly one Python. To get
the syntax right we need to use Jinja.

Unfortunately that means we need to consolidate our other `skip`s into
this one, which includes the Windows CUDA 11.8 skip. So we do so and try
to make the syntax as pleasing as we can within the constraints.
automerged PR by conda-forge/automerge-action
@jakirkham jakirkham requested a review from hcho3 as a code owner July 26, 2024 08:59
@jakirkham jakirkham merged commit 9b6bce7 into rapidsai:main Jul 26, 2024
@jakirkham jakirkham deleted the merge_upstream_13 branch July 31, 2024 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants